Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

tests: bluetooth: tester: increase BT RX stack size for nrf54l #21046

Merged
merged 1 commit into from
Mar 21, 2025
Merged

tests: bluetooth: tester: increase BT RX stack size for nrf54l #21046

merged 1 commit into from
Mar 21, 2025

Conversation

m-alperen-sener
Copy link
Contributor

nrf54l15 needs more BT RX stack size for the following test cases: MMDL/SR/GPL/BV-14-C
MMDL/SR/LHSL/BV-07-C
MMDL/SR/LHSL/BV-08-C
MMDL/SR/LXYL/BV-08-C
MMDL/CL/GPR/BV-08-C
MMDL/SR/LLN/BV-17-C

@m-alperen-sener
tests: bluetooth: tester: increase BT RX stack size for nrf54l
70889b0 
nrf54l15 needs more BT RX stack size for the following test cases:
MMDL/SR/GPL/BV-14-C
MMDL/SR/LHSL/BV-07-C
MMDL/SR/LHSL/BV-08-C
MMDL/SR/LXYL/BV-08-C
MMDL/CL/GPR/BV-08-C
MMDL/SR/LLN/BV-17-C

Signed-off-by: alperen sener <alperen.sener@nordicsemi.no>
@m-alperen-sener m-alperen-sener requested review from carlescufi and a team as code owners March 19, 2025 15:57
@github-actions github-actions bot added the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Mar 19, 2025
@NordicBuilder
Copy link
Contributor

NordicBuilder commented Mar 19, 2025 •
edited
Loading

CI Information

To view the history of this post, clich the 'edited' button above
Build number: 1

Inputs:

Sources:

sdk-nrf: PR head: 70889b035fc20963802819ba41ad6f2ddc1724f0

more details

sdk-nrf:

PR head: 70889b035fc20963802819ba41ad6f2ddc1724f0
merge base: 71866f3768c24605a95a698792ccb03ce46958d5
target head (main): 403069892efdcc8dad1df44978e029a4df48c406
Diff

Github labels

Enabled Name Description
ci-disabled Disable the ci execution
ci-all-test Run all of ci, no test spec filtering will be done
ci-force-downstream Force execution of downstream even if twister fails
ci-run-twister Force run twister
ci-run-zephyr-twister Force run zephyr twister
List of changed files detected by CI (1) 
tests
│  ├── bluetooth
│  │  ├── tester
│  │  │  ├── boards
│  │  │  │  │ nrf54l15dk_nrf54l15_cpuapp.conf

Outputs:

Toolchain

Version: 4ffa2202d5
Build docker image: docker-dtr.nordicsemi.no/sw-production/ncs-build:4ffa2202d5_e579f9fbe6

Test Spec & Results: ✅ Success; ❌ Failure; 🟠 Queued; 🟡 Progress; ◻️ Skipped; ⚠️ Quarantine

  • ◻️ Toolchain - Skipped: existing toolchain is used
  • ✅ Build twister
    • sdk-nrf test count: 469
  • ✅ Integration tests
Disabled integration tests
    • desktop52_verification
    • doc-internal
    • test_ble_nrf_config
    • test-fw-nrfconnect-apps
    • test-fw-nrfconnect-ble_mesh
    • test-fw-nrfconnect-ble_samples
    • test-fw-nrfconnect-boot
    • test-fw-nrfconnect-chip
    • test-fw-nrfconnect-fem
    • test-fw-nrfconnect-nfc
    • test-fw-nrfconnect-nrf-iot_cloud
    • test-fw-nrfconnect-nrf-iot_libmodem-nrf
    • test-fw-nrfconnect-nrf-iot_mosh
    • test-fw-nrfconnect-nrf-iot_positioning
    • test-fw-nrfconnect-nrf-iot_samples
    • test-fw-nrfconnect-nrf-iot_serial_lte_modem
    • test-fw-nrfconnect-nrf-iot_thingy91
    • test-fw-nrfconnect-nrf-iot_zephyr_lwm2m
    • test-fw-nrfconnect-nrf_crypto
    • test-fw-nrfconnect-proprietary_esb
    • test-fw-nrfconnect-ps
    • test-fw-nrfconnect-rpc
    • test-fw-nrfconnect-rs
    • test-fw-nrfconnect-tfm
    • test-fw-nrfconnect-thread
    • test-fw-nrfconnect-zigbee
    • test-low-level
    • test-sdk-audio
    • test-sdk-dfu
    • test-sdk-find-my
    • test-sdk-mcuboot
    • test-sdk-pmic-samples
    • test-sdk-sidewalk
    • test-sdk-wifi
    • test-secdom-samples-public

Note: This message is automatically posted and updated by the CI

@sonarqubecloud SonarQubeCloud
Copy link

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

PavelVPV
PavelVPV approved these changes Mar 20, 2025
View reviewed changes
Copy link
Contributor

@PavelVPV PavelVPV left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do you know why stack size is increased on nRF54L15?

@m-alperen-sener
Copy link
Contributor Author

m-alperen-sener commented Mar 20, 2025 •
edited
Loading

Do you know why stack size is increased on nRF54L15?

MBEDTLS is the cause the additional stack need. The actual PSA stack requirement will be investigated. Also, I haven't check what type of load or network activity is requiring stack increase particularly for those test cases.

@nordicjm nordicjm merged commit 692c84b into nrfconnect:main Mar 21, 2025
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@PavelVPV PavelVPV PavelVPV approved these changes

@alxelax alxelax alxelax approved these changes

@carlescufi carlescufi Awaiting requested review from carlescufi carlescufi is a code owner

Assignees
No one assigned
Labels
changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@m-alperen-sener @NordicBuilder @PavelVPV @alxelax @nordicjm